Skip to content

Add detailed GeoIP database path validation#6848

Open
Sungyoun-Kim wants to merge 1 commit into
opensearch-project:mainfrom
Sungyoun-Kim:fix-geoip-path-validation-errors
Open

Add detailed GeoIP database path validation#6848
Sungyoun-Kim wants to merge 1 commit into
opensearch-project:mainfrom
Sungyoun-Kim:fix-geoip-path-validation-errors

Conversation

@Sungyoun-Kim

@Sungyoun-Kim Sungyoun-Kim commented May 11, 2026

Copy link
Copy Markdown

Fixes #5865.

Summary:

  • Add class-level validation for MaxMind database paths.
  • Return path-specific messages for missing files, directories, unsupported HTTP endpoints, and mixed source types.
  • Add validation coverage for detailed messages.

Signed-off-by: Peter Kim <peter@rtzr.ai>
Co-authored-by: Codex (AI-generated) <codex@rtzr.ai>
@Sungyoun-Kim Sungyoun-Kim force-pushed the fix-geoip-path-validation-errors branch from c4f5d9d to f8efcd9 Compare May 11, 2026 05:18
@Sungyoun-Kim

Copy link
Copy Markdown
Author

Hi maintainers, friendly bump on this PR.

This addresses #5865 by replacing the generic database_paths validation failure with path-specific validation messages.

Could one code owner please take a look when available?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide detailed, path-specific validation errors in MaxMindDatabaseConfig.java

1 participant